preview of future fre.cmor#842
Closed
ilaflott wants to merge 51 commits into
Closed
Conversation
Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/3f296191-8cf1-4391-b69c-a844460e0681 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
- In test_remap_pp_components.py: replaced 3 nccmp -d calls with xr.testing.assert_equal, renamed test_nccmp_* to test_compare_* - In test_split_netcdf.py: replaced nccmp -d with xr.testing.assert_equal for data tests, replaced nccmp -mg with xr.testing.assert_identical for metadata tests Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/288da653-c4d3-4b41-bc3b-d062092ae272 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/288da653-c4d3-4b41-bc3b-d062092ae272 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/a4520c47-dde7-47e8-88b3-99780a457aa6 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/a4520c47-dde7-47e8-88b3-99780a457aa6 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
…ecks Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/b75a9df0-9832-414c-89bd-10bc365552bf Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
…sing netCDF4+numpy Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/b75a9df0-9832-414c-89bd-10bc365552bf Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Fix logic for file comparison results in test.
…led in a different PR
Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/afae15f4-790e-4d99-a369-d75a9644e2f4 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
…ion warnings, remove from required deps Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/83cd5af3-5071-4657-ba27-a32d87e1ded5 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/83cd5af3-5071-4657-ba27-a32d87e1ded5 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/83cd5af3-5071-4657-ba27-a32d87e1ded5 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
…er, update tests and pylintrc Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/1c5f6605-f239-4dd0-94b7-53b5458889fd Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
…er weighted average Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/1fb856bc-30a4-4be7-bb77-b2f42b051266 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
…ghted seas/month parametrized cases Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/ac9c29b8-e26b-4463-804e-8f22154798d3 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
…ix docstring Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/ac9c29b8-e26b-4463-804e-8f22154798d3 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
…65_day aliases) Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/757f704b-98a8-4a6a-8099-0810c9981656 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/96192d5d-8b4a-4d7a-95da-a0b43ed4da37 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
…atches table format Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/5f7a982a-1ccb-4536-94ea-694984b5bd0d Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
…le fetching Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/53b90d4a-98f0-4fdd-8ace-b3af2d66dd5a Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/53b90d4a-98f0-4fdd-8ace-b3af2d66dd5a Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/4ee8800b-bc98-4e4e-a90f-5c8bd95b757d Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/462398e4-aa55-4443-82c4-ca9dc8f90993 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Agent-Logs-Url: https://github.com/NOAA-GFDL/fre-cli/sessions/2b30185b-b512-44b8-9bb3-e95419553509 Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
…m-tests' into prerelease-frecmor
…t-remap-pp-components' into prerelease-frecmor
…tests' into prerelease-frecmor
…-use' into prerelease-frecmor
…t-config' into prerelease-frecmor
…p7' into prerelease-frecmor
…-input' into prerelease-frecmor
…s together as a prerelease for the frecmor submodule
Member
Author
|
first observation, |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #842 +/- ##
==========================================
- Coverage 84.12% 83.95% -0.17%
==========================================
Files 71 73 +2
Lines 4944 5098 +154
==========================================
+ Hits 4159 4280 +121
- Misses 785 818 +33
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Member
Author
|
can re-open this if needed. closing |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Describe your changes
the branch/fork is a merge of multiple in-development features across branches in other PRs. which ones is tracked in the script
to_merge.sh. this PR is not actually for merging/review, it is for previewing how things forfre.cmorare coming together, and having the assoc. pipelines provide feedback.currently merged into this branch :
fre.cmor: replacenccmpsubprocess calls withnetCDF4in cmor run subtool tests #826 /origin/copilot/remove-replace-nccmp-from-testsnccmpwithxarrayintest_remap_pp_components.py#827 /origin/copilot/remove-replace-nccmp-test-remap-pp-componentsnccmpdependency fromtest_split_netcdf.py, replace withxarrayandnumpy#828 /origin/copilot/remove-replace-nccmp-in-testsorigin/copilot/deprecate-cdo-python-cdo-usefre.cmor: newfre cmor initcommand for config fetching #834 /origin/copilot/add-empty-user-experiment-config(needs another unit test for coverage)fre.cmor: raise informative error when mip_era config mismatches table format (CMIP6 vs CMIP7) #836 /origin/copilot/fix-cmor-error-cmip6-cmip7fre.cmor: accept CF calendar aliases as equivalent (noleap / 365_day, etc.) #837 /origin/copilot/fix-flexibility-calendar-inputfre.cmor: update to3.14.2from3.14.0#817 /fork/frecmor-updates-for-cmip7todo, more important to check, direct influence on
fre.cmor:origin/copilot/deprecate-cdo-python-cdo-usefre.cmor: newfre cmor initcommand for config fetching #834 /origin/copilot/add-empty-user-experiment-config(needs another unit test for coverage)merge in
fre.cmor: improved and comprehensive omission tracking #833 /copilot/add-omission-tracking-featurefre.cmor: create CMIP7 flavored tests of cmor_run_subtool_further_examples #838 /copilot/create-cmip7-flavored-testsfre.cmor: harden branded-variable disambiguations #832 /copilot/harden-branded-variable-disambiguationstodo, lower priority, parallel to
fre.cmoror trivially relevant:log_and_raisehelper to mergefre_logger.errorandraisesteps #831 /copilot/merge-fre-logger-error-raisefre.maketests to avoidgetcwd, and lightpylintcleanup #808 /copilot/fix-flakey-unit-testscopilot/add-fre-version-to-model-yamluramirez8707/platform_yaml(note, a fork that is not mine)